Conversation
Co-authored-by: GeekTrainer <6109729+GeekTrainer@users.noreply.github.com>
Co-authored-by: GeekTrainer <6109729+GeekTrainer@users.noreply.github.com>
Co-authored-by: GeekTrainer <6109729+GeekTrainer@users.noreply.github.com>
Co-authored-by: GeekTrainer <6109729+GeekTrainer@users.noreply.github.com>
…user guidance on feature implementation and testing.
…y branch naming conventions
GeekTrainer
requested review from
chrisreddington and
peckjon
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR contains a comprehensive workshop rework and update that modernizes the structure, streamlines content, and enhances testing infrastructure. The changes consolidate two workshops into a single focused DevOps experience while updating all packages to their newest versions.
Key Changes:
- Removed the 1-hour Copilot workshop and consolidated into a single full-day DevOps workshop
- Added comprehensive E2E testing infrastructure with Playwright
- Enhanced development scripts and automation
- Updated all package versions to latest
Reviewed Changes
Copilot reviewed 61 out of 83 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| content/README.md | Updated to single workshop structure focusing on DevOps with GitHub |
| server/tests/test_app.py | Added new test for 404 error handling on dog endpoint |
| client/e2e-tests/ | Complete E2E testing suite with Playwright tests for all user workflows |
| scripts/ | Enhanced development scripts for environment setup, testing, and app startup |
| .github/workflows/ | New CI/CD workflows for automated testing |
| client/src/components/ | Added comprehensive test identifiers for reliable E2E testing |
Files not reviewed (1)
- client/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)
server/app.py:1
- These comment lines appear to be leftover development markers and should be removed for clean production code.
import os
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
@GeekTrainer many of us are using this 1 hour workshop today for the VS Code Dev Days through the end of the month. Do you have a backup we could use here or when do you plan on merging this in? https://github.com/github-samples/agents-in-sdlc seems good, but it is missing things around vs code and copilot that this one does today. |
|
@jamesmontemagno As an update, we're taking another look at this update, and it'll likely get scrapped. We're looking to use the workshop for Ignite, and have a meeting on Wednesday to talk about the approach we want to take. I'll make sure to keep you updated on whatever we decide. |
Several changes are included in this PR: